Added paginated response support for listing work items - #56
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a length-aware paginated return type for listing work items, so callers can page through results while also seeing pagination metadata (total/per-page/offset).
Changes:
- Introduces
LengthAwarePaginatedResponseto carry items + pagination metadata. - Updates
ManagesWorkItems::workitems()to return a paginated response and to page by slicing WIQL-returned IDs. - Updates
WorkItemsTestexpectations to assert the new response type and adds coverage for custom WIQL limits.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
src/Actions/ManagesWorkItems.php |
Changes workitems() to return a paginated response and adds offset/WIQL limit parameters. |
src/Responses/LengthAwarePaginatedResponse.php |
New response DTO for items + pagination metadata (total, perPage, offset, currentPage). |
tests/WorkItemsTest.php |
Updates tests to expect LengthAwarePaginatedResponse and adds a custom WIQL limit test. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.